-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support ESLint v7 #190
Support ESLint v7 #190
Conversation
BREAKING CHANGE: Support minimum ESLint version is v7.
BREAKING CHANGE: drop ESLint v6 support
BREAKING CHANGE: Enable new rules in @typescript-eslint/recommended - @typescript-eslint/no-extra-non-null-assertion - @typescript-eslint/no-extra-semi - @typescript-eslint/no-non-null-asserted-optional-chain -⚠️ @typescript-eslint/prefer-as-const
You would see the warning while installing this config.
But |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Thank you! |
ESLint v7 has been released so we'd like to support ESLint v7.
https://eslint.org/blog/2020/05/eslint-v7.0.0-released
THIS PR IS A BREAKING CHANGE because this drops to support ESLint v6.
Tasks
@typescript-eslint/eslint-plugin
ESLint v7 typescript-eslint/typescript-eslint#1550@typescript-eslint/parser
ESLint v7 typescript-eslint/typescript-eslint#1550eslint-plugin-jsx-a11y
https://github.com/evcohen/eslint-plugin-jsx-a11y/issues/683eslint-plugin-prettier
Add tests for running ESLint 7 prettier/eslint-plugin-prettier#291eslint-plugin-react
[New] support eslint v7 jsx-eslint/eslint-plugin-react#2635eslint-plugin-react-hooks
feat(eslint-plugin-react-hooks): Support ESLint 7.x facebook/react#18878